Les résultats de la boucle factorielle sont incorrects après la 5e itération

Vous dépassez la capacité du type int (2 147 483 647), donc votre résultat revient à la valeur int minimale. Essayez plutôt d'utiliser long.
Cela dit, la méthode que vous utilisez actuellement ne donnera pas la bonne réponse: en fait, vous calculez actuellement 10! ^ 2.
Pourquoi compliquer les choses? Vous pouvez facilement faire quelque chose comme ceci:
long x = 1L;
pour (int n = 1; n <10; n ++)
{
x * = n;
System.out.println (x);
}
1
2
6
24
120
720
5040
40320
362880
qui montre des factorielles successives jusqu'à 10! est atteint.
De plus, comme d'autres l'ont mentionné, si vous avez besoin de valeurs plus grandes que ce que peut prendre en charge longtemps, vous devez utiliser BigInteger, qui prend en charge la précision arbitraire.
|
C'est un problème de débordement d'entier. Utilisez long ou unsigned long au lieu de int. (Et comme @Dunes l'a suggéré, votre meilleur pari est vraiment BigInteger lorsque vous travaillez avec de très grands nombres, car il ne débordera jamais, théoriquement)
L'idée de base est que les entiers signés stockent les nombres entre -2 147 483 648 et 2 147 483 647, qui sont stockés sous forme de bits binaires (toutes les informations d'un ordinateur sont stockées sous forme de 1 et de 0)
Les nombres positifs sont stockés avec 0 dans le bit le plus significatif, et les nombres négatifs sont stockés avec 1 dans le bit le plus significatif. Si votre nombre positif devient trop grand en représentation binaire, les chiffres seront reportés sur le bit signé et transformeront votre nombre positif en représentation binaire d'un nombre négatif.
Ensuite, lorsque le factoriel devient plus grand que ce qu'un int non signé peut stocker, il «s'enroule» et perd le report de son bit le plus significatif (signé) - c'est pourquoi vous voyez le modèle d'alternance parfois des valeurs positives et négatives dans votre sortie.
|
Votre formule pour la factorielle est incorrecte. Ce que vous aurez, c'est ceci:
Étape 1: n * (n-1) = 10 * 9 = 90 => x = 1 * 90 = 90
Étape 2: n * (n-1) = 9 * 8 = 72 => x = 90 * 72 = 6480 ou, cela devrait être: 10 * 9 * 8 => 720
Mais les mauvais résultats viennent du fait que vous avez atteint la valeur maximale du type int comme indiqué par d'autres
Votre code doit être
classe publique Factorial
{
public static void main (String [] args)
{
factorielle double = 1;
pour (int n = factoriel; n> = 1; n--)
{
factoriel = factoriel * n;
System.out.printf ("% d", factoriel);
}
}
}
|
En plus de ce que les autres réponses mentionnent à propos du débordement, votre algorithme factoriel est également incorrect. dix! devrait calculer 10 * 9 * 8 * 7 * 6 * 5 * 4 * 3 * 2 * 1, vous faites (10 * 9) * (9 * 8) * (8 * 7) * (7 * 6) *. ..
Essayez de changer votre boucle comme suit:
int x = 1;
pour (int n = 10; n> 1; n--)
{
x = x * n;
System.out.printf ("% d", x);
}
Vous finirez par déborder si vous essayez de calculer la factorielle des nombres plus élevés, mais int est suffisamment grand pour calculer la factorielle de 10.
|
Ta Réponse
StackExchange.ifUsing ("éditeur", fonction () {
StackExchange.using ("externalEditor", function () {
StackExchange.using ("extraits", function () {
StackExchange.snippets.init ();
});
});
}, "extraits de code");
StackExchange.ready (fonction () {
var channelOptions = {
tags: "" .split (""),
id: "1"
};
initTagRenderer ("". split (""), "" .split (""), channelOptions);
StackExchange.using ("externalEditor", function () {
// Doit lancer l'éditeur après les extraits, si les extraits sont activés
if (StackExchange.settings.snippets.snippetsEnabled) {
StackExchange.using ("extraits", function () {
createEditor ();
});
}
autre {
createEditor ();
}
});
function createEditor () {
StackExchange.prepareEditor ({
useStacksEditor: faux,
heartbeatType: 'réponse',
autoActivateHeartbeat: faux,
convertImagesToLinks: vrai,
noModals: vrai,
showLowRepImageUploadWarning: vrai,
reputationToPostImages: 10,
bindNavPrevention: vrai,
suffixe: "",
imageUploader: {
brandingHtml: "Powered by \ u003ca href = \" https: //imgur.com/ \ "\ u003e \ u003csvg class = \" svg-icon \ "width = \" 50 \ "height = \" 18 \ "viewBox = \ "0 0 50 18 \" fill = \ "none \" xmlns = \ "http: //www.w3.org/2000/svg \" \ u003e \ u003cpath d = \ "M46.1709 9.17788C46.1709 8.26454 46.2665 7.94324 47.1084 7.58816C47.4091 7.46349 47.7169 7.36433 48.0099 7.26993C48.9099 6.97997 49.672 6.73443 49.672 5.93063C49.672 5.22043 48.9832 4.61182 48.1414 4.61182C47.4335 4.61182 46.72556.9762.6943 4.61182C47.4335 4.61182 46.72554.91628 46.094 4.68.4335 4.61182 46.72554.91628 46.094.49.48.4335 4.61182 46.72554.91628 46.094.49.48.4335 4.61182 46.7256 4.9762.698 43.1481 6.59048V11.9512C43.1481 13.2535 43.6264 13.8962 44.6595 13.8962C45.6924 13.8962 46.1709 13.2535 46.1709 11.9512V9.17788Z \ "/ \ u003e \ u003cpath d = \" M32.492 10.1419C32.492 12.6954.0 34184.01448.0 14.6954.0 34184.0 34184.0 14.6954.0 34184.0 14484.0 14.6954.0 34184.0 14484.0 41,5985 12,6954 41,5985 10,1419V6,59049C41,5985 5,28821 41,1394 4,66232 40,1061 4,66232C39,0732 4,66232 38,5948 5,28821 38,5948 6,59049V9,60062C38,5948 10,8521 38,2696 11,5455 37,0451 11,4455,5C35,820 521 35.4954 9.60062V6.59049C35.4954 5.28821 35.0173 4.66232 34.0034 4.66232C32.9703 4.66232 32.492 5.28821 32.492 6.59049V10.1419Z \ "/ \ u003e \ u003cpath fill-rule = \" evenodd \ "clip-rule = \" evenodd \ "clip-rule = \" evenodd \ "d. = \ "M25.6622 17.6335C27.8049 17.6335 29.3739 16.9402 30.253715.6379C30.8468 14.7755 30.9615 13.5579 30.9615 11.9512V6.59049C30.9615 5.28821 30.4833 4.66231 29.4502 4.66231C28.9913 4.66231 28.4555 4.94978 28.1109 5.50789C27.499 4.86533 26.7335 4.5608725.9615 5.28821 30.4833 4.66231 29.4502 4.66231C28.9913 4.66231 28.4555 4.94978 28.1109 5.50789C27.499 4.86533 26.7335 4.56087 25.723005 13.913C26.5612 13,913 27,4607 13,4902 28,1109 12,7229 28,1161 12,7799 12.6616C28.1109 28,121 12.8346C28.1256 12,8854 28,1301 12,9342 28,1301 14,4373 27,2502 15,2321 12.983C28.1301 25,777 15.2321C24.8349 15,2321 24,1352 14,9821 23,5661 22,8472 14,5218 14.7787C23.176 14,6393 22,5437 14.5218C21 .7977 14.5218 21.2429 15.0123 21.2429 15.6887C21.2429 16.7375 22.9072 17.6335 25.6622 17.6335ZM24.1317 9.27932C24.1317 7.94324 24.9928 7.09766 26.1024 7.09766C27.2119 7.09766 28.0918 7.94324.1311.51.2119 7.09766 28.0918 7.94324.1311.51.2119 7,09766 28.0918 7.94324.1311.51.218 9,2241 24115 9,22417 11,6171 269171 269176 11,6179 9.27932Z \ "/ \ u003e \ u003cpath d = \" M16.8045 11.9512C16.8045 13.2535 17.2637 13.8962 18.2965 13.8962C19.3298 13.8962 19.8079 13,2535 19.8079 11.9512V8.12928C19.8079 5.82936 18.4879 4.62866 16.4027 4.62866C15.1594 4.62866 14.279 4.98375 13.3609 5.88013C12.653 5.05154 11.6581 4.62866 10.3573 4.62866C9.34336 4.62866 8.57809 59931 7.9466 4.62866 8.57809 59931 7.9466 4.62866 8.57809 59931 7.9466 4.62866 8.57809 59931 7.9466 5,28821 5.00066 6.59049V11.9512C5.00066 13.2535 5.47873 13.8962 6.51203 13.8962C7.54479 13.8962 8.0232 13.2535 8.0232 11.9512V8.90741C8.0232 7.58817 8.44431 6.91179 9.53458 6.91179C10.5104 6.91162.893 13.8119.893 13.81179.105 12.893 13.8119.403 13.8119.893 .4375 13.8962 13.9157 13.2535 13.9157 11.9512V8.90741C13.9157 7.58817 14.3365 6.91179 15.4269 6.91179C16.4027 6.91179 16.8045 7.58817 16.8045 8.94108V11.9512Z \ "/ \ u003e \ u00390Cpath d = 2,8755,216,3755,216,35,38,35,35,35,35,35,35,35,35,35,35,35,35,35,35,35,35,35,35,35,35,38 et 8,94108V11,9512Z 4.66232C0.791758 4.66232 0.313354 5.28821 0.313354 6.59049V11.9512C0.313354 13.2535 0.791758 13.8962 1.82471 13.8962C2.85798 13.8962 3.31675 13.2535 3.31675 11.9512V6.59049Z \ "/ \ u003e \ u 003cpath d = \ "M1.87209 0.400291C0.843612 0.400291 0 1.1159 0 1.98861C0 2.87869 0.822846 3.57676 1.87209 3.57676C2.90056 3.57676 3.7234 2.87869 3.7234 1.98861C3.7234 1.1159 2.90056 0.400291" Remplissage 1 1.87209 / \ u003e \ u003c / svg \ u003e \ u003c / a \ u003e ",
contentPolicyHtml: "Contributions des utilisateurs sous licence \ u003ca href = \" https: //stackoverflow.com/help/licensing \ "\ u003ecc by-sa \ u003c / a \ u003e \ u003ca href = \" https://stackoverflow.com / legal / content-policy \ "\ u003e (politique de contenu) \ u003c / a \ u003e",
allowUrls: vrai
},
onDemand: vrai,
discardSelector: ".discard-answer"
, immédiatementShowMarkdownHelp: true, enableTables: true, enableSnippets: true
});
}
});
Merci d'avoir répondu à Stack Overflow!
Veuillez vous assurer de répondre à la question. Fournissez des détails et partagez vos recherches!
Mais évitez…
Demander de l'aide, des éclaircissements ou répondre à d'autres réponses.
Faire des déclarations basées sur des opinions; les sauvegarder avec des références ou une expérience personnelle.
Pour en savoir plus, consultez nos conseils sur la rédaction de bonnes réponses.
Brouillon enregistré
Brouillon rejeté
Inscrivez-vous ou connectez-vous
StackExchange.ready (fonction () {
StackExchange.helpers.onClickDraftSave ('# login-link');
});
Inscrivez-vous avec Google
Inscrivez-vous via Facebook
Inscrivez-vous par e-mail et mot de passe
Soumettre
Publier en tant qu'invité
Nom
Email
Obligatoire, mais jamais affiché
StackExchange.ready (
fonction () {
StackExchange.openid.initPostLogin ('. New-post-login', 'https% 3a% 2f% 2fstackoverflow.com% 2fquestions% 2f13772608% 2Les résultats-de-boucle-factuelle-sont-incorrects-après-la-cinquième-itération% 23new- answer ',' question_page ');
}
);
Publier en tant qu'invité
Nom
Email
Obligatoire, mais jamais affiché
Publiez votre réponse
Jeter
En cliquant sur «Publier votre réponse», vous acceptez nos conditions d'utilisation, notre politique de confidentialité et notre politique de cookies
Ce n'est pas la réponse que vous recherchez? Parcourez les autres questions marquées java ou posez votre propre question.